Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent multiple changelog commits at once #222

Merged
merged 2 commits into from
Aug 18, 2023

Conversation

DebugOk
Copy link
Contributor

@DebugOk DebugOk commented Aug 16, 2023

About the PR

Prevent multiple changelog actions from running at once via job concurrency. This means they get queued instead of all ran at once. If there are ever any future actions that will commit directly to the repository, they should also run under the commit_action group.

Changelog

No CL no fun

If several try running at once, it'll fail for every action started while another one was running.
@DebugOk
Copy link
Contributor Author

DebugOk commented Aug 16, 2023

I should note it'll still make the first one fail because you modified the repo, but at least it wont cause several failures :clueless:

@DebugOk DebugOk marked this pull request as draft August 16, 2023 22:32
Just makes it that much more robust
@DebugOk DebugOk marked this pull request as ready for review August 16, 2023 22:42
@DebugOk
Copy link
Contributor Author

DebugOk commented Aug 16, 2023

I just slapped in a pull origin/master before it tries to push the new changelog. Should mean it'll never fail unless git does some weird shit or the script has a stroke and dies

@Cheackraze Cheackraze merged commit 7466081 into new-frontiers-14:master Aug 18, 2023
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants